-
Notifications
You must be signed in to change notification settings - Fork 501
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update our Codecov setup #3190
Update our Codecov setup #3190
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #3190 +/- ##
==========================================
+ Coverage 79.12% 79.14% +0.02%
==========================================
Files 131 131
Lines 14173 14173
Branches 2971 2971
==========================================
+ Hits 11214 11217 +3
Misses 2118 2118
+ Partials 841 838 -3 |
…hon into antonpirker/codecov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
…hon into antonpirker/codecov
This PR: - Cleans up our pytest configuration and moves everything into pytest.ini - Make our Codecov setup more valuable. If there is now a test failing we get a comment in the PR showing the failing test and its output.
This PR:
pytest.ini
If there is now a test failing we get this in the PR:
Fixes #3194